Skip to content

Isolate thread replies into a rip-out-friendly feature module - #3

Merged
Overtorment merged 1 commit into
masterfrom
isolate-thread-replies
Aug 6, 2026
Merged

Isolate thread replies into a rip-out-friendly feature module#3
Overtorment merged 1 commit into
masterfrom
isolate-thread-replies

Conversation

@GladosBlueWallet

Copy link
Copy Markdown

Summary

  • Move Phase A (thread replies, settled suppress, markers) into src/thread-replies/ with a small public API
  • Keep src/review/ as core PR review only — no thread-reply knowledge
  • Document rip-out steps in AGENTS.md (swap CLI wiring, delete the folder)

Test plan

  • npm test (18 passing)
  • npm run typecheck
  • Confirm CLI still wires processPrReviewWithThreadReplies / processPrThreadReplies
  • Rip-out smoke check: mentally verify deleting src/thread-replies/ + switching CLI to processPrReview leaves core review intact

Made with Cursor

Move Phase A out of core review so the feature can be replaced or deleted without touching the PR review path.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Overtorment
Overtorment merged commit 50e28df into master Aug 6, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants